Skip to content

Adds code fixes to compile successfully and pass tests#4

Merged
tonyqiu123 merged 10 commits intomainfrom
third-attempt
Jan 30, 2026
Merged

Adds code fixes to compile successfully and pass tests#4
tonyqiu123 merged 10 commits intomainfrom
third-attempt

Conversation

@ericahan22
Copy link
Member

Notion ticket link

N/A

Implementation description

  • Commented out/fixed code that was "broken" due to deprecated packages and not using built-in user auth/file storage at project setup

Steps to test

  1. Run the application with docker-compose up --build
  2. Run docker exec -it scv2_ts_backend sh -c "node migrate up"
  3. Run pip install pytest python-dotenv inflection --user
  4. Run tests: cd e2e-tests then python -m pytest

What should reviewers focus on?

  • Let me know what errors you run into or if I should make any modifications to the Steps to test

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@ericahan22 ericahan22 requested a review from tonyqiu123 January 29, 2026 23:40
@tonyqiu123 tonyqiu123 merged commit 817826e into main Jan 30, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants